explicit parse_settings = TRUE#706
explicit parse_settings = TRUE#706pascal-sauer wants to merge 1 commit intoREditorSupport:masterfrom
Conversation
|
|
||
| if (file.exists(path)) { | ||
| lints <- lintr::lint(path, cache = cache, text = content) | ||
| lints <- lintr::lint(path, cache = cache, text = content, parse_settings = TRUE) |
There was a problem hiding this comment.
I think we can omit this diff? the change in {lintr}, IIRC, is about the latter invocation (no path, just text=).
There was a problem hiding this comment.
You are probably right about this, would need to double check to be sure :)
There was a problem hiding this comment.
I just had the same problem and had to modify a local copy of languageserver to fix my .linter file not being used (actually I was going to submit a PR myself).
I can confirm this change is necessary even in this case. :)
There was a problem hiding this comment.
In my opinion, it makes sense to always be explicit to protect from future lintr changes.
|
I'm unsure how the maintainers (I guess @renkun-ken @randy3k) want to proceed with this PR. Do you want me to bring this PR into a state so it can be merged, and what steps would that entail? Or would you rather implement a fix yourself (totally fine by me!)? |
|
@ManuelHentschel Sorry for bothering you, but could you take a look at this? |
|
In the absence of updates, see a workaround in: |
5 participants
According to https://github.com/r-lib/lintr/releases with lintr 3.3.0-1:
This means
.lintrfiles are ignored for the linting vscode does in the background by default. Explicitly settingparse_settings = TRUE(as suggested in this PR) helps.